Skip to content

[stable33] feat: file list filters#7019

Merged
vitormattos merged 26 commits intostable33from
backport/7018/stable33
Feb 26, 2026
Merged

[stable33] feat: file list filters#7019
vitormattos merged 26 commits intostable33from
backport/7018/stable33

Conversation

@backportbot-libresign
Copy link
Copy Markdown

@backportbot-libresign backportbot-libresign Bot commented Feb 26, 2026

Backport of #7018

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

…leListFilterChips in filters slot

Signed-off-by: Vitor Mattos <[email protected]>
onFilterUpdateChips was emitting libresign:filters:update before
filter_status was persisted, causing getAllFiles to run with stale
state. The subsequent call from onFilterUpdateChipsAndSave (which
actually persists the new value) was then skipped because loading=true.

Only onFilterUpdateChipsAndSave should trigger a re-fetch, since it is
the action that actually updates filter_status/filter_modified.

Signed-off-by: Vitor Mattos <[email protected]>
PHP stores and reads filter config under files_list_filter_status
and files_list_filter_modified (after migration Version17001Date20260210).
The JS was using the old short keys in both loadState reads and PUT
requests, so saved values were never read back on page load.

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos marked this pull request as ready for review February 26, 2026 21:50
@vitormattos vitormattos merged commit 7ffc039 into stable33 Feb 26, 2026
63 of 65 checks passed
@vitormattos vitormattos deleted the backport/7018/stable33 branch February 26, 2026 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 0. Needs triage

Development

Successfully merging this pull request may close these issues.

1 participant